chore: apply checkstyle again #1339
Conversation
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #1339 +/- ##
============================================
- Coverage 79.06% 78.85% -0.22%
Complexity 651 651
============================================
Files 74 74
Lines 2489 2497 +8
Branches 257 257
============================================
+ Hits 1968 1969 +1
- Misses 441 448 +7
Partials 80 80
☔ View full report in Codecov by Sentry. |
|
LGTM. I ran Checkstyle on this branch, and there were no other changes triggered. |
|
Sonarqube reliability lint in examples - I have renamed the sonarqube config file on the batch branch so that we exclude examples from the copy-paste analysis. You might like to do the same here to get past the quality gate. The previous config file name was incorrect for the analysis mode we are using and had no effect. |
|
Kudos, SonarCloud Quality Gate passed!
|
Issue #, if available:
Description of changes:
Looks like the latest version of Intellij or the checkstyle plugin for intellij does not format the same way than before (when I pushed my checkstyle code I was on a previous version of IntelliJ).
And looks like some formatted code has been unformatted since I pushed (bad merges or something).
So I apply it again.
@scottgerring @mriccia @eldimi @kozub @msailes @AlexeySoshin, please make sure to have checkstyle plugin installed with the latest version of IntelliJ and apply it to your changes (not to the whole project). See contributing guide. When merging main into your branches, please make sure to apply the formatting changes too.
Checklist
Breaking change checklist
RFC issue #:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.